Skip to content

[pull] main from bazel-contrib:main#344

Merged
pull[bot] merged 4 commits into
garymm:mainfrom
bazel-contrib:main
Aug 2, 2025
Merged

[pull] main from bazel-contrib:main#344
pull[bot] merged 4 commits into
garymm:mainfrom
bazel-contrib:main

Conversation

@pull

@pull pull Bot commented Aug 2, 2025

Copy link
Copy Markdown

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.3)

Can you help keep this open source service alive? 💖 Please sponsor : )

dotoleeoak and others added 4 commits August 1, 2025 23:07
Closes #2985

Suppose invalid `experimental_index_url_overrides` in `pip.parse` is set
like below.

```bzl
pip.parse(
    experimental_index_url = "https://pypi.org/simple",
    experimental_index_url_overrides = {"mypy": "https://invalid.com"},
    hub_name = "pypi",
    requirements_lock = "//:requirements_lock.txt",
)
```

It fails as follows, showing only "pypi.org" as pypi index url, not
"invalid.com" for for `mypy` package.

```
Error in fail: Failed to download metadata for ["mypy"] for from urls: ["https://pypi.org/simple"].
If you would like to skip downloading metadata for these packages please add 'simpleapi_skip=["mypy"]' to your 'pip.parse' call.
```

To show overridden url for each package, show url of each package that
has been failed to download metadata. The error message with this PR is
like below.

```
Error in fail:
Failed to download metadata of the following packages from urls:
{
    "mypy": "https://invalid.com",
}

If you would like to skip downloading metadata for these packages please add 'simpleapi_skip=["mypy"]' to your 'pip.parse' call.
```
Fixes #2976. The rest of it was fixed in #3045.
… follow convention (#3128)

Prefix `experimental_allow_relative_imports` with `python_` to match the
rest of the directives.

1.6.0 hasn't been released yet, so this is a non-breaking change.
I noticed that the CI error messages for #3114 are not useful. This
patch aims to help with that by printing the output of the REPL code.
If there's an exception during startup for example, then the test log
will now contain the stack trace.

---------

Co-authored-by: Ignas Anikevicius <240938+aignas@users.noreply.github.com>
@pull pull Bot locked and limited conversation to collaborators Aug 2, 2025
@pull pull Bot added the ⤵️ pull label Aug 2, 2025
@pull pull Bot merged commit 6819b84 into garymm:main Aug 2, 2025
1 check passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants